[#162] TodoInfoSheetView에 생성일, 완료일을 보여주도록 추가한다#167
Conversation
opficdev
commented
Mar 3, 2026
opficdev
commented
- closed TodoDetailContentView에 생성일을 볼 수 있도록 추가한다 #162
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 TodoInfoSheetView의 정보 표시 기능을 확장하여 사용자가 할 일의 생성일과 완료일을 한눈에 확인할 수 있도록 개선합니다. 이는 할 일의 생명주기에 대한 더 명확한 이해를 돕고, 사용자 경험을 향상시키는 데 기여합니다. Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| HStack(spacing: 8) { | ||
| Image(systemName: "calendar") | ||
| .foregroundStyle(.white) | ||
| Text(createdAt.formatted(date: .abbreviated, time: .omitted)) | ||
| Spacer() | ||
| } | ||
| .padding(.vertical, 10) | ||
| .padding(.horizontal, 12) | ||
| .background( | ||
| RoundedRectangle(cornerRadius: 12) | ||
| .fill(.blue) | ||
| ) |
There was a problem hiding this comment.
생성일(createdAt)을 표시하는 부분의 스타일링에 몇 가지 개선점이 있습니다.
- 텍스트 색상:
Image에는.foregroundStyle(.white)가 적용되었지만, 날짜를 표시하는Text에는 적용되지 않았습니다. 파란색 배경 위에 기본 텍스트 색상이 표시되면 가독성이 떨어질 수 있으므로,Text에도.foregroundStyle(.white)를 추가하여 일관성을 맞추는 것이 좋습니다. - 하드코딩된 색상: 배경색으로
.fill(.blue)가 하드코딩되어 있습니다. '마감일' 섹션에서Color(.tertiarySystemFill)을 사용한 것처럼, 다크 모드 등 다양한 환경에 더 잘 대응할 수 있는 시맨틱 컬러(Semantic Color)를 사용하거나 Asset에 색상을 정의하여 사용하는 것을 권장합니다..fill(.green)(81행)도 마찬가지입니다.
| HStack(spacing: 8) { | |
| Image(systemName: "calendar") | |
| .foregroundStyle(.white) | |
| Text(createdAt.formatted(date: .abbreviated, time: .omitted)) | |
| Spacer() | |
| } | |
| .padding(.vertical, 10) | |
| .padding(.horizontal, 12) | |
| .background( | |
| RoundedRectangle(cornerRadius: 12) | |
| .fill(.blue) | |
| ) | |
| HStack(spacing: 8) { | |
| Image(systemName: "calendar") | |
| .foregroundStyle(.white) | |
| Text(createdAt.formatted(date: .abbreviated, time: .omitted)) | |
| .foregroundStyle(.white) | |
| Spacer() | |
| } | |
| .padding(.vertical, 10) | |
| .padding(.horizontal, 12) | |
| .background( | |
| RoundedRectangle(cornerRadius: 12) | |
| .fill(.blue) | |
| ) |
| Text( | ||
| completedAt? | ||
| .formatted(date: .abbreviated, time: .omitted) | ||
| ?? "완료하지 않음" |
There was a problem hiding this comment.
뷰 내부에 "완료하지 않음"과 같은 문자열이 하드코딩되어 있습니다. 이러한 문자열은 Localizable.xcstrings 파일에 추가하여 지역화(Localization)를 지원하도록 하는 것이 좋습니다.
또한, Localizable.xcstrings에는 "완료 시점"으로 추가되었지만, 뷰에서는 "완료일"(63행)을 사용하고 있어 불일치가 발생합니다. 일관성을 위해 Localizable.xcstrings에 정의된 키를 사용해주세요. "마감일 없음"(49행) 문자열도 마찬가지로 지역화가 필요합니다.
| ?? "완료하지 않음" | |
| ?? String(localized: "완료하지 않음") |
1 participant
